Skip to content

Fix: update gh token - #43

Merged
vaibhav-jain-exp merged 3 commits into
mainfrom
udpate-gh-token
Jul 24, 2026
Merged

Fix: update gh token#43
vaibhav-jain-exp merged 3 commits into
mainfrom
udpate-gh-token

Conversation

@vaibhav-jain-exp

@vaibhav-jain-exp vaibhav-jain-exp commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Note

Medium Risk
Changes release automation credentials and git push behavior; a misconfigured or under-scoped GH_PUBLISH_TOKEN could break releases or grant broader repo write access than intended.

Overview
Updates the release workflow so release commits and tags are pushed with a dedicated GH_PUBLISH_TOKEN instead of the default GITHUB_TOKEN.

Checkout now sets persist-credentials: false, and the Commit Release step configures a temporary GitHub HTTP Authorization header from GH_PUBLISH_TOKEN before git push / git push --tags, then removes it. Release commit/tag messaging is driven via a VERSION env var (same input as before).

Reviewed by Cursor Bugbot for commit 9eff25e. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is ON. A cloud agent has been kicked off to fix the reported issue.

Reviewed by Cursor Bugbot for commit 52a3bb0. Configure here.

Comment thread .github/workflows/release.yaml Outdated
actions/checkout persists the default GITHUB_TOKEN in local git config by
default. Setting GITHUB_TOKEN in a later step env block does not override
those persisted credentials.

Disable persist-credentials on checkout and configure the origin remote
with GH_PUBLISH_TOKEN before pushing release commits and tags.

Co-authored-by: Vaibhav Jain <vaibhav-jain-exp@users.noreply.github.com>
Comment thread .github/workflows/release.yaml
zhukaihan
zhukaihan previously approved these changes Jul 24, 2026
Avoid interpolating github context directly in run scripts. Use env
vars for VERSION and authenticate via http.extraheader instead of
embedding the token in the git remote URL.

Co-authored-by: Vaibhav Jain <vaibhav-jain-exp@users.noreply.github.com>
@vaibhav-jain-exp
vaibhav-jain-exp merged commit a74ac32 into main Jul 24, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants